Skip to content

Conversation

@jkobject
Copy link
Contributor

Describe your changes

moving this to a new PR

Copy link
Member

@rcannood rcannood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor comments

Comment on lines -54 to -62
# TODO: Try to find working installation of flash attention (flash-attn<1.0.5)
setup:
#- type: python
# pypi:
# - gdown
# - scgpt # Install from PyPI to get dependencies
#- type: docker
# # Force re-installing from GitHub to get bug fixes
# run: pip install --upgrade --no-deps --force-reinstall git+https://github.com/bowang-lab/scGPT.git
Copy link
Member

@rcannood rcannood Oct 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# TODO: Try to find working installation of flash attention (flash-attn<1.0.5)
setup:
#- type: python
# pypi:
# - gdown
# - scgpt # Install from PyPI to get dependencies
#- type: docker
# # Force re-installing from GitHub to get bug fixes
# run: pip install --upgrade --no-deps --force-reinstall git+https://github.com/bowang-lab/scGPT.git
setup:

This shouldn't be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why?

type: integer
description: The maximum length of the gene sequence.
default: 4000
default: 2300
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was already merged, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

then I don't see why it still shows as a modification... :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants